-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(datasource): add debian datasource #30071
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution, and the well documented functionality!
This is my first pass as there is a lot going on here and we have to be sure that the caching is setup efficiently.
Would this work for every APT repository?
We should start simple and then start to optimize. |
Thx for the initial review, I will fix the findings soon.
should work with every apt repository, which follows the https://wiki.debian.org/DebianRepository standard. |
Co-authored-by: Sebastian Poxhofer <[email protected]>
…t/debian-datasource
renovatebot#30974) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…t#30981) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
….0 (renovatebot#30983) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ot#30986) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…enovatebot#30994) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…enovatebot#30995) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Kriese <[email protected]>
@oxdev03 Thanks for the contribution and pushing this through! 🚀 |
🎉 This issue has been resolved in version 38.57.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Changes
Context
Resolves: #7041, #24906
Concept
Based on the discussion and the reused implementation (see the related issue for more details):
Evaluation of Provided or Default Registry URL
https://ftp.debian.org/debian?release=bullseye&components=main,contrib&binaryArch=amd64
is evaluated to:.../debian/dists/bullseye/main/binary-amd64
.../debian/dists/bullseye/contrib/binary-amd64
Check for Existing Compressed Package
Packages.gz
file has already been downloaded and extracted:Iterate through Package Index
Return relevant release information
Documentation (please check one with an [x])
included documentation for new datasource
How I've tested my work (please select one)
I have verified these changes via:
Real Test
Open Points
Does the current caching implementation for the datasource need to be reworked? Can the Renovate team provide a concept/details for this? (e.g., invalidation of the compressed file based on its timestamp)
Is the current approach generic enough? Are there any points that were not considered?
If significant rework is required, please let me know in advance.